Skip to content

Master mysterious egg loco 1 #4489

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 7, 2025

Conversation

loco-odoo
Copy link

No description provided.

@robodoo
Copy link

robodoo commented Apr 25, 2025

This PR targets the un-managed branch odoo-dev/odoo:master-mysterious-egg, it needs to be retargeted before it can be merged.

@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch 2 times, most recently from 5906a60 to 1540589 Compare April 28, 2025 08:44
this.dialogService = this.services.dialog;
const editableEls = findOEditable(this.editable);
this.elToTranslationInfoMap = new Map();
const translatedAttrs = ["placeholder", "title", "alt", "value"];
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: for the moment, there is a problem when a form field has an input attribute. I will solve it in another PR; the problem is that interactions that should not be started in edit mode are somehow started and destroyed when entering translate mode.

@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch from 1540589 to 4674893 Compare April 29, 2025 07:24
@@ -0,0 +1,72 @@
import { Dialog } from "@web/core/dialog/dialog";
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just copy pasted from website

@loco-odoo
Copy link
Author

loco-odoo commented Apr 29, 2025

To fix in stable:

  • Add a title to an image
  • Translate the title with a special character (example: "<")
  • Click on "Ok"
  • Try to retranslate the attribute => it is encoded as &lt;

Task: https://www.odoo.com/odoo/project/49/tasks/4759395

@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch from 4674893 to bdc5c86 Compare April 29, 2025 08:43
@@ -37,9 +49,6 @@ export class SetupEditorPlugin extends Plugin {
el.setAttribute("data-editor-message", _t("DRAG BUILDING BLOCKS HERE"));
}
});

// Set the `contenteditable` attribute on the editables.
this.setContenteditable();
Copy link
Author

@loco-odoo loco-odoo Apr 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done at the normalize so useless at the end of the setup

@ged-odoo ged-odoo force-pushed the master-mysterious-egg branch from e24556e to 897f4ab Compare April 29, 2025 18:30
@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch 5 times, most recently from 5ebb06b to 715ffd2 Compare May 5, 2025 08:02
@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch from 715ffd2 to 1c2df02 Compare May 6, 2025 10:06
@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch 2 times, most recently from c1d7a40 to 35d5f63 Compare May 7, 2025 09:58
loco-odoo added 2 commits May 7, 2025 14:24
- translate attributes
- translate selections
- add a message when trying to translate not editable elements
- add a message to explain the translation colors
@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch from 35d5f63 to faa967c Compare May 7, 2025 12:24
@FrancoisGe FrancoisGe marked this pull request as ready for review May 7, 2025 12:28
@FrancoisGe FrancoisGe merged commit c9f1567 into master-mysterious-egg May 7, 2025
@robodoo
Copy link

robodoo commented May 7, 2025

This PR targets the un-managed branch odoo-dev/odoo:master-mysterious-egg, it needs to be retargeted before it can be merged.

@FrancoisGe FrancoisGe deleted the master-mysterious-egg-loco-1 branch May 7, 2025 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants